-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(analyse_prometheus): store errors instead of exit #237
base: main
Are you sure you want to change the base?
Conversation
Fixes grafana#236 Signed-off-by: Furkan <[email protected]> Co-authored-by: Emin <[email protected]> Co-authored-by: Yasin <[email protected]> Co-authored-by: Batuhan <[email protected]>
Kind ping here 🤞 @gotjosh @gouthamve |
@Dentrax something's changed with our CLA setup. Asking within the org, to try and find out. |
@Dentrax it's due to an issue we're having with CLA Assistant :/ Our administrator is in the process of trying to resolve it. |
@Dentrax the CLA Assistant issue should be solved now, but it might still be slow in picking up your having signed (it is for me). |
Kind ping 🤞 @aknuds1 |
Signed-off-by: ShuzZzle <[email protected]>
Fixes #236
Make sure you have
metrics-in-grafana.json
file in the directory.Signed-off-by: Furkan [email protected]
Co-authored-by: Emin [email protected]
Co-authored-by: Yasin [email protected]
Co-authored-by: Batuhan [email protected]